Skip to content

Conversation

@Bahadir67
Copy link
Contributor

@Bahadir67 Bahadir67 commented Jan 25, 2026

Summary

  • Add Turkish (tr) to the language selection list in SettingsPopup.svelte
  • Turkish translations already exist in all lang/tr.json files across the codebase

Changes

  • Added { id: 'tr', label: ui.string.Turkish, logo: '🇹🇷' } to the langs array

Screenshots

The Turkish flag emoji (🇹🇷) will appear in the language selector alongside existing languages.

Test Plan

  • Verify Turkish appears in language dropdown
  • Verify selecting Turkish changes UI to Turkish
  • Verify all UI strings display correctly in Turkish

🤖 Generated with Claude Code

@huly-github-staging
Copy link

Connected to Huly®: UBERF-15303

@Bahadir67 Bahadir67 force-pushed the add-turkish-language branch 2 times, most recently from 693c0f2 to a6ac14a Compare January 26, 2026 05:33
Added Turkish (Türkçe) to the language selection dropdown in SettingsPopup.svelte
and added the Turkish IntlString definition to plugin.ts.
Turkish translation files already exist in the codebase (lang/tr.json files).

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Signed-off-by: Bahadır Atılgan <gokcel.bahadir@gmail.com>
@Bahadir67 Bahadir67 force-pushed the add-turkish-language branch from 2337ef6 to 66534f3 Compare January 28, 2026 04:21
Bahadir67 and others added 3 commits January 28, 2026 08:00
Adds a GitHub Actions workflow to build and push the front container
image to GitHub Container Registry for testing Turkish language support.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Signed-off-by: Bahadır Atılgan <gokcel.bahadir@gmail.com>
Signed-off-by: Bahadır Atılgan <gokcel.bahadir@gmail.com>
Signed-off-by: Bahadır Atılgan <gokcel.bahadir@gmail.com>
@Bahadir67
Copy link
Contributor Author

Could you please approve the workflow run? This PR adds Turkish language support to the settings popup. Turkish translation files already exist in the codebase (lang/tr.json files), this PR just enables the language option in the UI.

@BykhovDenis
Copy link
Member

@Bahadir67 Why we need build-front-tr.yml?

@Bahadir67
Copy link
Contributor Author

You're right, the build-front-tr.yml was added for my local testing purposes. I'll remove it from this PR. The actual changes are only in the two UI files (SettingsPopup.svelte and plugin.ts).

Signed-off-by: Bahadır Atılgan <gokcel.bahadir@gmail.com>
@BykhovDenis
Copy link
Member

Thank you for your contribution

@Bahadir67
Copy link
Contributor Author

Thank you for the review and approval! Happy to contribute to Huly. Looking forward to seeing Turkish language support in the next release.

@ArtyomSavchenko
Copy link
Member

Hi @Bahadir67
Thank you for your contribution
Do you also plan to add the missing Turkish translations?
For example, if you need help with this, I could generate translations using AI, but I don't have the knowledge of the Turkish language to verify this.

@Bahadir67
Copy link
Contributor Author

Yes! The Turkish translations are mostly complete (65/67 files). I noticed plugins/inbox-assets/lang/tr.json is missing. I can add that and also review existing translations for accuracy. AI-generated translations would be helpful as a starting point, and I can verify them.

Added missing Turkish translation file for the inbox-assets plugin.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Signed-off-by: Bahadır Atılgan <gokcel.bahadir@gmail.com>
@ArtyomSavchenko
Copy link
Member

Yes! The Turkish translations are mostly complete (65/67 files). I noticed plugins/inbox-assets/lang/tr.json is missing. I can add that and also review existing translations for accuracy. AI-generated translations would be helpful as a starting point, and I can verify them.

I think I can then merge the current changes, and the missing translations can be added in separate PR.
Thank you once again for your contribution.

@ArtyomSavchenko ArtyomSavchenko merged commit 4563fab into hcengineering:develop Jan 29, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants